Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix RCLC int parameter get (cherry-pick) #272

Merged
merged 5 commits into from
Apr 1, 2022

Conversation

JanStaschulat
Copy link
Contributor

@JanStaschulat JanStaschulat commented Mar 23, 2022

git cherry-pick fd0c9a5d91155d908acd6af893ce9d60bf0d1a0e
see PR #269

Signed-off-by: Pablo Garrido <[email protected]>

Update

Signed-off-by: Pablo Garrido <[email protected]>

Fix test

Signed-off-by: Pablo Garrido <[email protected]>

Update

Signed-off-by: Pablo Garrido <[email protected]>

Update examples

Signed-off-by: Pablo Garrido <[email protected]>
@JanStaschulat
Copy link
Contributor Author

Error on Open Robotics Build farm due to timeout:

1: [ RUN      ] ActionServerTest.goal_accept
1/6 Test #1: rclc_test ........................***Timeout  60.05 sec

@pablogs9 pablogs9 closed this Mar 25, 2022
@pablogs9 pablogs9 reopened this Mar 25, 2022
JanStaschulat and others added 3 commits April 1, 2022 15:26
* updated documentation: bloom release table for rolling ubuntu yammy for all packages.

Signed-off-by: Jan Staschulat (CR/ADA1.2) <[email protected]>

* install phyhton dependendcy for ubuntu yammi transition - should be removed later.

Signed-off-by: Jan Staschulat (CR/ADA1.2) <[email protected]>

* interactive installation

Signed-off-by: Jan Staschulat (CR/ADA1.2) <[email protected]>

* removed installation of python3.9

Signed-off-by: Jan Staschulat (CR/ADA1.2) <[email protected]>

* using docker_image: ubuntu:jammy

Signed-off-by: Jan Staschulat (CR/ADA1.2) <[email protected]>

* using ubuntu-20.04

Signed-off-by: Jan Staschulat (CR/ADA1.2) <[email protected]>

* [email protected]

Signed-off-by: Jan Staschulat (CR/ADA1.2) <[email protected]>

* retry

Signed-off-by: Jan Staschulat (CR/ADA1.2) <[email protected]>

* retry

Signed-off-by: Jan Staschulat (CR/ADA1.2) <[email protected]>

* update text

Signed-off-by: Jan Staschulat (CR/ADA1.2) <[email protected]>

* commented out unit tests ActionServerTest

Signed-off-by: Jan Staschulat (CR/ADA1.2) <[email protected]>

* uncrustify

Signed-off-by: Jan Staschulat (CR/ADA1.2) <[email protected]>

* removed dependencies.repos

Signed-off-by: Jan Staschulat (CR/ADA1.2) <[email protected]>

* removed vcs-repo-file-url from ci.yml

Signed-off-by: Jan Staschulat (CR/ADA1.2) <[email protected]>

* revert ActionServerTest

Signed-off-by: Jan Staschulat (CR/ADA1.2) <[email protected]>

* uncrustify

Signed-off-by: Jan Staschulat (CR/ADA1.2) <[email protected]>

* reverted changes in test_action_server.cpp

Signed-off-by: Jan Staschulat (CR/ADA1.2) <[email protected]>
Signed-off-by: Pablo Garrido <[email protected]>

Update

Signed-off-by: Pablo Garrido <[email protected]>

Fix test

Signed-off-by: Pablo Garrido <[email protected]>

Update

Signed-off-by: Pablo Garrido <[email protected]>

Update examples

Signed-off-by: Pablo Garrido <[email protected]>
@JanStaschulat JanStaschulat merged commit 449575c into master Apr 1, 2022
@JanStaschulat JanStaschulat deleted the fix/rolling-int-parameter-get branch April 1, 2022 13:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants